Javascript function loops too many times and returns undefined